Skip to content

Fix build and warnings - #78

Merged
techomancer merged 3 commits into
techomancer:mainfrom
danifunker:fix-build-and-warnings
Aug 8, 2026
Merged

Fix build and warnings#78
techomancer merged 3 commits into
techomancer:mainfrom
danifunker:fix-build-and-warnings

Conversation

@danifunker

Copy link
Copy Markdown
Contributor
  • builds were fixed as I was working on it, so it was rebased
  • remove all compilation warnings

danifunker and others added 3 commits August 8, 2026 09:31
2be036c ported the GUI's live MIPS estimate onto CyclesPtr, so the worker now
holds a raw pointer into the Machine's MipsCore. Every path that drops the
machine has to clear it first — a use-after-free the compiler can't catch.

Co-Authored-By: Claude Opus 5 (1M context) <[email protected]>
- winit 0.30: EventLoop::run is deprecated, so Ui::run's closure becomes a
  UiApp ApplicationHandler driven by run_app. Pure mechanical move —
  run_app is literally run() with a dispatch closure, same events.
- raw-window-handle 0.6: HasRawWindowHandle/raw_window_handle() are
  deprecated; use HasWindowHandle + window_handle().as_raw().
- unused mut in mips_cache_v2 (only opcodefusion writes prev_s1) and in
  main.rs (only the pcap prompt writes cfg).
- coffdump's Fdr/Pdr mirror on-disk records: allow(dead_code).
- drop the unused SOCKET_FILTERS const in iris-gui.

Co-Authored-By: Claude Opus 5 (1M context) <[email protected]>
--features r5k,r5ksc_triton has not compiled since the Triton L2-enable sync
landed: `cache` needs to be mut for set_l2_enabled. Only that feature writes
it, so the mut is paired with a cfg'd allow(unused_mut).

Also silence unreachable_code on idle_state, which follows the jit
dispatch's return when both features are on (iris-gui's premiere build).

Co-Authored-By: Claude Opus 5 (1M context) <[email protected]>
@techomancer
techomancer merged commit 321dac0 into techomancer:main Aug 8, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants